This repository was archived by the owner on Feb 21, 2022. It is now read-only.
Open
Conversation
Squashes more deprecation warnings.
No derivative tabulations yet.
Banish expensive sympy and use fast numpy.
Performance could be improved further, but it is already much faster than Lagrange elements. (Although, Lagrange elements are less relevant for high-order than Bernstein elements.)
Add Bernstein element Approved-by: Lawrence Mitchell <wence@gmx.li> Approved-by: rckirby <robert_kirby@baylor.edu>
Remove reliance on numpy.matrix Approved-by: Chris Richardson <chris@bpi.cam.ac.uk>
updates for latest pytest * Updates for pytest version * Update pipelines Approved-by: Garth Wells <gnw20@cam.ac.uk>
* toriesz: Address review comments Fix a typo and flake8 Add some documentation for to_riesz make point dict and deriv dict loops look alike in to_riesz bug fix in derivatives for to_riesz fix typo Address dham's concerns on dual_set.py remove some code, comments pep 8, remove some old code Switch to new to_riesz method general/abstract to_riesz, lift to dual Merged in chris/update-pytest (pull request #53)
Discontinuous Serendipity element for cuboids (DPC)
This was referenced Jun 10, 2019
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
-The previous coordinate change mapping stretched out the simplex that the nodes reside on, which is undesirable.
-The reference element was passed to DualSet as a simplex, it is now passed as the original hypercube
These bug fixes are introduced in this pull request.